From: Jasper St. Pierre <jstpierre@mecheye.net>
Date: Thu, 25 Apr 2013 20:32:01 +0000 (-0400)
Subject: network: Don't create submenus for multiple-connection items
X-Git-Tag: archive/raspbian/50.5-1+rpi1~1^2~1013^2^2~4063
X-Git-Url: https://dgit.raspbian.org/%22http://www.example.com/cgi/%22/%22http:/www.example.com/cgi/%22?a=commitdiff_plain;h=a55288bda06fc7043a220409e61a3bab6e82fa09;p=gnome-shell.git

network: Don't create submenus for multiple-connection items

As multiple-connections for a Wi-Fi AP won't fit in the new design,
remove submenus right now. Simply make a simple item that connects
to the first known connection for the AP, which should be the common
case.

https://bugzilla.gnome.org/show_bug.cgi?id=698918
---

diff --git a/js/ui/status/network.js b/js/ui/status/network.js
index 06317bbb0..6538ab5c8 100644
--- a/js/ui/status/network.js
+++ b/js/ui/status/network.js
@@ -1118,23 +1118,8 @@ const NMDeviceWireless = new Lang.Class({
                         break;
 
                     if (network.item) {
-                        if (network.item instanceof PopupMenu.PopupSubMenuMenuItem) {
-                            let items = network.item.menu._getMenuItems();
-                            if (items.length == 2) {
-                                // we need to update the connection list to convert this to a normal item
-                                forceupdate = true;
-                            } else {
-                                for (let j = 0; j < items.length; j++) {
-                                    if (items[j]._connection.get_uuid() == connection.get_uuid()) {
-                                        items[j].destroy();
-                                        break;
-                                    }
-                                }
-                            }
-                        } else {
-                            network.item.destroy();
-                            network.item = null;
-                        }
+                        network.item.destroy();
+                        network.item = null;
                     }
                     break;
                 }
@@ -1198,14 +1183,7 @@ const NMDeviceWireless = new Lang.Class({
         }
 
         if(network.connections.length > 0) {
-            if (network.connections.length == 1) {
-                network.item = this._createAPItem(network.connections[0], network, false);
-            } else {
-                let title = network.ssidText;
-                network.item = new PopupMenu.PopupSubMenuMenuItem(title);
-                for (let i = 0; i < network.connections.length; i++)
-                    network.item.menu.addMenuItem(this._createAPItem(network.connections[i], network, true));
-            }
+            network.item = this._createAPItem(network.connections[0], network, false);
         } else {
             network.item = new NMNetworkMenuItem(network.accessPoints[0]);
             network.item.connect('activate', Lang.bind(this, function() {